Skip to content

[hardware] sipeed slogic new hardware support#212

Closed
taorye wants to merge 4 commits intosigrokproject:masterfrom
taorye:master
Closed

[hardware] sipeed slogic new hardware support#212
taorye wants to merge 4 commits intosigrokproject:masterfrom
taorye:master

Conversation

@taorye
Copy link

@taorye taorye commented Mar 10, 2023

As the title says, a new hardware want to be supported in mainline.
If there is something I do not prepared or missed, please give me some advice and I will as soon as possible to handle it.

@taorye
Copy link
Author

taorye commented Mar 10, 2023

$9 for SLogic Lite8, $14.9 for SLogic Combo8(LA+4UART+CKLink+DAPLink)

SLogic Lite8 (Logic Analyzer)

channel: 8CH (MAX)
samplerate: 160Msps (MAX)
bandwidth: 320Mb/s (MAX)

supported configuration for continuously sampling

  • 160M@2CH
  • 80M@4CH
  • 40M@8CH

input signal level: 0~3.6V (TTL)

Copy link
Author

@taorye taorye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have reviewed.

@MartinHerren
Copy link

MartinHerren commented Oct 4, 2023

Nice to see a patch for this analyzer, have just received mine (early order) and finally found time to test it.
Tested it including your 0bdfe4ceb03c74fe03c07ba620210ddd00fde4fe commit on your hardware-sipeed-slogic-support branch

Hope to soon see it mainlined into libsigrok.

Currently still running into some issues:

  • starting an acquisition sometimes completely freezes libsigrok
  • needs to run as root. Fixed on my branch on commit 24978df3a369380466fcb20ced79fae074909e55
  • missing dependency on libusb. Fixed on my branch on commit dc604b7fcabec9ff158bb0de1dec7a7bdc76a7ff
  • when switching to a sample rate of 64 or 80 MS/s, the number of channels gets correctly reduced to 4, the same when switching to 128 or 160 MS/s it gets reduced to 2. Switching back to 64 or 80 MS/s again gets back to 4 channels. Switching to 40 MS/s or lower won't give back all 8 channels. The device needs to get opened again. Fixed on my branch on commit fe587691af2f86089385a0b145cb2bf2f2e30f5e

Thanks for your work and i'm looking forward to test this device some more !

SR_DRIVER([serial DMM], [serial-dmm], [serial_comm])
SR_DRIVER([serial LCR], [serial-lcr], [serial_comm])
SR_DRIVER([Siglent SDS], [siglent-sds])
SR_DRIVER([Sipeed Slogic Analyzer], [sipeed-slogic-analyzer])

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a libsigrok developer, but shouldn't a dependency to libusb be added here ?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm, this is needed to prevent it from trying to build and fail if libusb is not available

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are right, libusb should be added.
Thanks :)

@ANAU-designs
Copy link

Very interesting driver for this new very little logic analyzer (and more...)
Would be nice to also add the udev rules, update [contrib/60-libsigrok.rules]

@@ -312,6 +312,9 @@ ATTRS{idVendor}=="10c4", ATTRS{idProduct}=="ea60", ENV{ID_SIGROK}="1"
# UNI-T UT-D09 multimeter cable (for various UNI-T and rebranded DMMs)
ATTRS{idVendor}=="10c4", ATTRS{idProduct}=="ea80", ENV{ID_SIGROK}="1"

+# Sipeed SLogic Combo 8
+ATTRS{idVendor}=="359f", ATTRS{idProduct}=="0300", ENV{ID_SIGROK}="1"
+
# Sysclk LWLA1016
ATTRS{idVendor}=="2961", ATTRS{idProduct}=="6688", ENV{ID_SIGROK}="1"

@MartinHerren
Copy link

MartinHerren commented Oct 8, 2023

Very interesting driver for this new very little logic analyzer (and more...) Would be nice to also add the udev rules, update [contrib/60-libsigrok.rules]

@@ -312,6 +312,9 @@ ATTRS{idVendor}=="10c4", ATTRS{idProduct}=="ea60", ENV{ID_SIGROK}="1" # UNI-T UT-D09 multimeter cable (for various UNI-T and rebranded DMMs) ATTRS{idVendor}=="10c4", ATTRS{idProduct}=="ea80", ENV{ID_SIGROK}="1" +# Sipeed SLogic Combo 8 +ATTRS{idVendor}=="359f", ATTRS{idProduct}=="0300", ENV{ID_SIGROK}="1" + # Sysclk LWLA1016 ATTRS{idVendor}=="2961", ATTRS{idProduct}=="6688", ENV{ID_SIGROK}="1"

Yes, i already fixed that on my branch: master...MartinHerren:libsigrok:slogiclite8
Changes

  • Added udev rules
  • Fixed channel number selection to get again 8 channels when selecting 40 MS/s or less
  • Fixed build dependencies

I don't know if @taorye is still around and wants to update his PR or if we should open a new one.

There are still issues that the it regularly freezes and needs unplugging/replugging.

@ANAU-designs
Copy link

Very interesting driver for this new very little logic analyzer (and more...) Would be nice to also add the udev rules, update [contrib/60-libsigrok.rules]
@@ -312,6 +312,9 @@ ATTRS{idVendor}=="10c4", ATTRS{idProduct}=="ea60", ENV{ID_SIGROK}="1" # UNI-T UT-D09 multimeter cable (for various UNI-T and rebranded DMMs) ATTRS{idVendor}=="10c4", ATTRS{idProduct}=="ea80", ENV{ID_SIGROK}="1" +# Sipeed SLogic Combo 8 +ATTRS{idVendor}=="359f", ATTRS{idProduct}=="0300", ENV{ID_SIGROK}="1" + # Sysclk LWLA1016 ATTRS{idVendor}=="2961", ATTRS{idProduct}=="6688", ENV{ID_SIGROK}="1"

Yes, i already fixed that on my branch: master...MartinHerren:libsigrok:slogiclite8 Changes

* Added udev rules

* Fixed channel number selection to get again 8 channels when selecting 40 MS/s or less

* Fixed build dependencies

I don't know if @taorye is still around and wants to update his PR or if we should open a new one.

There are still issues that the it regularly freezes and needs unplugging/replugging.

Indeed there are a few issue[S] (I try to have it working under Linux) and it's rather strange behaviour in most of my tests, I do not have much time to investigate fully at the moment.

Regarding the PR, I do not know @taorye, and I am rather new to using GIT, was stuck to Subversion for some reasons :) so I do not know the best way to proceed, neither the relation between @taorye and Sipeed. There is an email address on her/his github page though.

I hope to free some time later this month to investigate more, let's see if this dream becomes some sort of reality

@MartinHerren
Copy link

Hej,

we'll see. If i have time to fix the usb issue and the original author didn't react, i'll probably open a new PR.

@taorye
Copy link
Author

taorye commented Nov 1, 2023

Thanks for your attention

Very interesting driver for this new very little logic analyzer (and more...) Would be nice to also add the udev rules, update [contrib/60-libsigrok.rules]
@@ -312,6 +312,9 @@ ATTRS{idVendor}=="10c4", ATTRS{idProduct}=="ea60", ENV{ID_SIGROK}="1" # UNI-T UT-D09 multimeter cable (for various UNI-T and rebranded DMMs) ATTRS{idVendor}=="10c4", ATTRS{idProduct}=="ea80", ENV{ID_SIGROK}="1" +# Sipeed SLogic Combo 8 +ATTRS{idVendor}=="359f", ATTRS{idProduct}=="0300", ENV{ID_SIGROK}="1" + # Sysclk LWLA1016 ATTRS{idVendor}=="2961", ATTRS{idProduct}=="6688", ENV{ID_SIGROK}="1"

Yes, i already fixed that on my branch: master...MartinHerren:libsigrok:slogiclite8 Changes

* Added udev rules

* Fixed channel number selection to get again 8 channels when selecting 40 MS/s or less

* Fixed build dependencies

I don't know if @taorye is still around and wants to update his PR or if we should open a new one.

There are still issues that the it regularly freezes and needs unplugging/replugging.

I'll come back soon. There are more to be updated before next year.

@yjun123
Copy link

yjun123 commented Jan 4, 2024

Hello, guys.

I have attempted to patch the mainline of libsigrok with MartinHerren's branch, specifically the slogiclite8 branch. I have tested it with the PulseView mainline.

I have encountered an issue where setting the sampling frequency to 5MHz or below can cause data acquisition to freeze. This issue is 100% reproducible.

In the case of a sampling rate greater than 5MHz, it works fine without any issues (?

my device is the Sipeed Slogic Combo 8.

@taorye taorye closed this by deleting the head repository Feb 1, 2024
@rkolbaskin
Copy link

So is it dead?

@mkroman
Copy link

mkroman commented Feb 22, 2024

So is it dead?

Certainly looks like it to me. There hasn't been a firmware release since 2023-09 for the SLogic Combo 8, and the provided PulseView AppImage hasn't been updated since 2023-08.. and I don't think there's another PR.

Looks a lot like vaporware tbh.. Hopefully development picks up again at some point. I don't understand why it wasn't at least made open source if there's not enough resources to work on it. :/

@g5pw
Copy link

g5pw commented Feb 22, 2024

Hm, I just noticed it actually uses the M0S module inside, apparently!

@MartinHerren
Copy link

Hej all,

Sorry for the late reply, didn't have time to use my Sipeed SLogic recently.

Juste rebased and rebuilt by branch (with @taorye's original support plus my fixes) on latest libsigrock's master branche.

Will try to use it during next week to try to reproduce @yjun123's issue and also my USB freezes.

I don't think it is a 'dead project' as they are still selling it. They haven't published new firmware and software as they might consider it works 'good enough' for them. Hopefully a stable version will end up officially in libsigrock.

I'm not even sure github is the correct place to open a PR for libsigrock as they never reacted to this PR. It is a read-only mirror but they say 'PR welcome'. For the moment i'll try to keep my branch alive and up to date with upstream to be able to continue to use my own device.

If it works stable enough and other people including upstream are interested i can open a new pull request.

My branch: https://github.com/MartinHerren/libsigrok/tree/slogiclite8, rebased on today's upstream:master.

@axel-h
Copy link

axel-h commented Jul 5, 2024

@MartinHerren since this PR is close due to @taorye deleting the branch, are you planning to open a new PR from your branch?

@0xDRRB
Copy link

0xDRRB commented Aug 17, 2024

Sipeed provides an AppImage binary version of Pulseview with a libsigrok modified to support this hardware, without releasing the source code. Isn't that a violation of the GPL?

@g5pw
Copy link

g5pw commented Sep 11, 2024

A request was made here sipeed/sipeed_wiki#698, I've actually just emailed support to request the source code.

@trabucayre
Copy link
Contributor

@0xDRRB, @g5pw : after this request, email and this post, @sipeed has opensourced his repository here.
Haven't compared the repo and this PR but I'm not sure there is more information to obtain...

@armandas
Copy link

armandas commented Jan 2, 2025

Confusingly, there are two branches in sipeed/sigrok_slogic with very different implementations. Can anyone tell from the code which one would be better suited for merging into the mainline?

EDIT: branch hardware-sipeed-slogic-support doesn't build, so I guess hardware-sipeed-slogic-analyzer-support then?

@ForsakenHarmony
Copy link

Seems like the repo is still getting updates

@taorye
Copy link
Author

taorye commented Apr 9, 2025

The process of integrating the latest developments into the main branch has officially commenced. Community members and contributors are invited to monitor the progress through the dedicated Pull Request sigrokproject/libsigrok#262. This PR serves as a pivotal point for tracking updates, facilitating discussions, and ensuring transparency in the development cycle.

@MartinHerren
Copy link

Oh, great news ! Thanks !

Sorry didn't have any time for follow up recently, still have my SLogic 8 and ready to try it again !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.